Skip to content

Conversation

@s-barannikov
Copy link
Contributor

@s-barannikov s-barannikov commented Sep 1, 2025

The generated disassembler can now correctly decode these instructions. All we need to do is to add bits<0> z to their encodings and provide a decoder method that adds Z register to the instruction.

@s-barannikov s-barannikov force-pushed the users/s.barannikov/decoder-operands-2-riscv branch from cc9f40c to 90b336f Compare September 1, 2025 18:19
@s-barannikov s-barannikov force-pushed the users/s.barannikov/decoder-operands-3-avr branch 2 times, most recently from 067f93e to 235b15d Compare September 1, 2025 19:11
@s-barannikov s-barannikov force-pushed the users/s.barannikov/decoder-operands-2-riscv branch 2 times, most recently from 9f30122 to aa16f8a Compare September 1, 2025 21:05
@s-barannikov s-barannikov force-pushed the users/s.barannikov/decoder-operands-3-avr branch from 235b15d to b1cd999 Compare September 1, 2025 21:05
@s-barannikov s-barannikov force-pushed the users/s.barannikov/decoder-operands-2-riscv branch from aa16f8a to fd7e685 Compare September 2, 2025 16:08
@s-barannikov s-barannikov force-pushed the users/s.barannikov/decoder-operands-3-avr branch 2 times, most recently from 6ff221b to d3e77cd Compare September 2, 2025 16:40
@s-barannikov s-barannikov force-pushed the users/s.barannikov/decoder-operands-2-riscv branch from fd7e685 to 4188fa4 Compare September 2, 2025 16:40
@s-barannikov s-barannikov force-pushed the users/s.barannikov/decoder-operands-2-riscv branch from 4188fa4 to d01399a Compare September 3, 2025 19:51
@s-barannikov s-barannikov force-pushed the users/s.barannikov/decoder-operands-3-avr branch 2 times, most recently from 1346d77 to f88e35a Compare September 3, 2025 20:17
@s-barannikov s-barannikov force-pushed the users/s.barannikov/decoder-operands-2-riscv branch from d01399a to a85a8d9 Compare September 3, 2025 20:17
@s-barannikov s-barannikov marked this pull request as ready for review September 3, 2025 21:11
@benshi001
Copy link
Member

It would be better to add an [NFC] in the title.

@s-barannikov s-barannikov changed the title [AVR] Remove workarounds for instructions using Z register [AVR] Remove workarounds for instructions using Z register (NFCI) Sep 4, 2025
Copy link
Member

@benshi001 benshi001 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@s-barannikov s-barannikov force-pushed the users/s.barannikov/decoder-operands-2-riscv branch from a85a8d9 to de1f705 Compare September 4, 2025 13:37
@s-barannikov s-barannikov force-pushed the users/s.barannikov/decoder-operands-3-avr branch from f88e35a to 55e234b Compare September 4, 2025 13:37
@s-barannikov s-barannikov force-pushed the users/s.barannikov/decoder-operands-3-avr branch from 55e234b to ff4fc23 Compare September 4, 2025 14:14
@s-barannikov s-barannikov force-pushed the users/s.barannikov/decoder-operands-2-riscv branch from de1f705 to 32e1a07 Compare September 4, 2025 14:14
@s-barannikov s-barannikov force-pushed the users/s.barannikov/decoder-operands-3-avr branch from ff4fc23 to e488057 Compare September 4, 2025 14:59
@s-barannikov s-barannikov force-pushed the users/s.barannikov/decoder-operands-2-riscv branch 2 times, most recently from d19b8cd to d564a29 Compare September 4, 2025 15:23
@s-barannikov s-barannikov changed the base branch from users/s.barannikov/decoder-operands-2-riscv to main September 4, 2025 15:24
The generated disassembler can now correctly decode these instructions.
@s-barannikov s-barannikov force-pushed the users/s.barannikov/decoder-operands-3-avr branch from e488057 to 15d65d9 Compare September 4, 2025 15:25
@s-barannikov s-barannikov merged commit 69d0c3e into main Sep 4, 2025
8 of 9 checks passed
@s-barannikov s-barannikov deleted the users/s.barannikov/decoder-operands-3-avr branch September 4, 2025 16:04
s-barannikov added a commit that referenced this pull request Sep 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants